Clean up agent profile — deduplicate info, add ERC-8004 tag#716
Merged
realproject7 merged 1 commit intomainfrom Apr 1, 2026
Merged
Clean up agent profile — deduplicate info, add ERC-8004 tag#716realproject7 merged 1 commit intomainfrom
realproject7 merged 1 commit intomainfrom
Conversation
- Remove Model/Genre/Registered from hero area (shown in Agent Identity card) - Remove Description from Agent Identity card (shown in hero bio) - Add ERC-8004 badge to Agent Identity card header (matches Pro badge style) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
project7-interns
approved these changes
Apr 1, 2026
Collaborator
project7-interns
left a comment
project7-interns
left a comment
There was a problem hiding this comment.
T2b APPROVE
Clean deduplication. All #713 criteria met:
- ✅ Removed Model/Genre/Registered from hero area (−16 lines of agent metadata)
- ✅ Removed Description from Agent Identity card (−6 lines)
- ✅ Added
ERC-8004badge:bg-accent/10 text-accent— matches Pro badge style - ✅ Agent Identity card retains: Agent ID, Name, Model, Genre, Registered
- ✅ Non-agent profiles unaffected (changes gated by
agentMeta) - ✅ Build passes
+4/−23, single file. No concerns.
project7-interns
approved these changes
Apr 1, 2026
Collaborator
project7-interns
left a comment
project7-interns
left a comment
There was a problem hiding this comment.
Verdict: APPROVE
Summary
This change matches issue #713 and cleanly removes duplicated agent metadata while keeping the Agent Identity card as the single source of truth.
Findings
- None.
Decision
Model, Genre, and Registered are no longer duplicated in the hero, Description stays only in the bio area, and the Agent Identity card now adds the requested ERC-8004 badge without affecting non-agent profiles. I am approving.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes #713
Self-Verification
ERC-8004tag visible on top right of Agent Identity cardProbadge on Farcaster cardnpm run buildpasses🤖 Generated with Claude Code